-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option for WLabel, WText and WDynamicLabel to have shadows #248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, the changes for WLabel
look good. Note that this PR doesn't fully address #233 since it's also about the other text widgets which don't support shadows either.
Now it does 😉 |
The PR doesn't compile. Could you fix the compilation errors? The other PR's build is also failing. |
@Juuxel could you take a look at this? I would like to use it in my mod 😉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just some minor doc typos/fixes.
src/main/java/io/github/cottonmc/cotton/gui/widget/WDynamicLabel.java
Outdated
Show resolved
Hide resolved
bdf2175
to
cb2d80e
Compare
Fixed |
@Juuxel sorry for pinging again, but could this be merged now that it's approved? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the PR!
Closes #245
Closes #233